Skip to content

fix go get path#355

Closed
laijs wants to merge 1 commit into
opencontainers:masterfrom
laijs:go-get
Closed

fix go get path#355
laijs wants to merge 1 commit into
opencontainers:masterfrom
laijs:go-get

Conversation

@laijs

@laijs laijs commented Mar 22, 2016

Copy link
Copy Markdown
Contributor

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

so we remove the "https://"

Signed-off-by: Lai Jiangshan jiangshanlai@gmail.com

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

so we remove the "https://"

Signed-off-by: Lai Jiangshan <jiangshanlai@gmail.com>
@wking

wking commented Mar 22, 2016

Copy link
Copy Markdown
Contributor

On Mon, Mar 21, 2016 at 07:25:25PM -0700, Lai Jiangshan wrote:

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

This is also covered by a79fdef (Makefile: add target for tools,
2016-03-17, #349), where it's a drive-by. I expect it doesn't matter
particularly much which PR lands the fix, but the sooner we land one
of them the less likely we'll be to get another duplicate ;).

@laijs

laijs commented Mar 23, 2016

Copy link
Copy Markdown
Contributor Author

thanks, close it

@laijs laijs closed this Mar 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants